Skip to content

Devp 660 modify api client error messages#57

Open
shihanpan wants to merge 3 commits intomainfrom
DEVP-660-modify-api-client-error-messages
Open

Devp 660 modify api client error messages#57
shihanpan wants to merge 3 commits intomainfrom
DEVP-660-modify-api-client-error-messages

Conversation

@shihanpan
Copy link
Collaborator

@shihanpan shihanpan commented Jan 27, 2026

No description provided.

@shihanpan
Improve 401 Unauthorized error message with permission guidance
052e2cc 
    - Clarify that 401 errors can be due to invalid/expired API token OR insufficient permissions
    - Add direct link to Workato API client management documentation
    - Restructure error message to show both possible causes and resolution steps
    - Improve actionability of error message for users debugging auth issues

    🤖 Generated with [Claude Code](https://claude.com/claude-code)

    Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Jan 27, 2026 •
edited
Loading

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py68494%10–11, 69–70
cli
   __init__.py52394%49, 51, 60
   containers.py270100% 
cli/commands
   __init__.py00100% 
   api_clients.py291996%27, 302, 448–449, 483, 493, 584, 615, 623
   api_collections.py257398%28, 183, 347
   assets.py47295%57–58
   connections.py526599%591, 593, 599, 637, 988
   data_tables.py165596%31, 253, 267, 321–322
   guide.py166199%106
   init.py1070100% 
   profiles.py2310100% 
   properties.py97198%21
   pull.py172298%193–194
   workspace.py39294%61, 71
cli/commands/connectors
   __init__.py00100% 
   command.py90297%110, 159
   connector_manager.py203498%176, 292, 300–301
cli/commands/projects
   __init__.py00100% 
   command.py2721096%359–362, 373, 439–441, 491, 495
   project_manager.py166795%48, 66, 263–264, 276, 317, 325
cli/commands/push
   __init__.py00100% 
   command.py133496%109, 112, 230, 308
cli/commands/recipes
   __init__.py00100% 
   command.py427997%117, 133–134, 272–275, 403, 709
   validator.py7062097%174, 883, 1136, 1223, 1246, 1279, 1281–1282, 1359–1361, 1457–1458, 1517–1518, 1707–1708, 1736–1738
cli/utils
   __init__.py40100% 
   exception_handler.py3655186%134–135, 140–141, 143–144, 174–175, 181, 184–185, 270, 300, 340, 359–363, 366–370, 372–374, 378–379, 382–384, 399–402, 435, 462, 489, 546, 581, 640, 642–643, 648–649, 651–655
   gitignore.py140100% 
   ignore_patterns.py230100% 
   spinner.py430100% 
   token_input.py451762%70, 98, 100, 102–104, 107, 124, 127–128, 132–133, 136, 139–140, 142, 144
   version_checker.py135695%24, 26, 33–34, 72, 102
cli/utils/config
   __init__.py50100% 
   manager.py5313393%127, 138, 149, 157–159, 162, 165, 177, 227–228, 399, 417, 421, 424–427, 443, 464, 478, 491, 533, 623–624, 661, 875, 1018–1019, 1033–1034, 1090, 1149
   models.py330100% 
   profiles.py3341595%99, 195–196, 199, 234–236, 261–263, 522, 530, 541–542, 546
   workspace.py680100% 
TOTAL584221596% 

@shihanpan shihanpan force-pushed the DEVP-660-modify-api-client-error-messages branch 2 times, most recently from e5faa1b to e1c6a2d Compare January 27, 2026 21:01
@shihanpan shihanpan self-assigned this Jan 28, 2026
@shihanpan shihanpan force-pushed the DEVP-660-modify-api-client-error-messages branch from cf44dad to 5b6771a Compare January 28, 2026 00:22
@shihanpan
separate the authz and authn message for all commands except in worka…
96d7527 
…to init as there is no way to reliably differentiate between them at that step
@shihanpan shihanpan force-pushed the DEVP-660-modify-api-client-error-messages branch from ecb6a0a to 96d7527 Compare January 28, 2026 00:37
@shihanpan shihanpan requested a review from cmworkato January 29, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cmworkato cmworkato Awaiting requested review from cmworkato

At least 1 approving review is required to merge this pull request.

Assignees

@shihanpan shihanpan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@shihanpan